home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c++-part2 / 15895 < prev    next >
Encoding:
Text File  |  1996-08-05  |  2.5 KB  |  96 lines
  1. Path: trib.apple.com!usenet
  2. From: Amir <Amir@bayarea.net>
  3. Newsgroups: comp.lang.c++
  4. Subject: Re: [HELP!!!] Segmentation Fault!
  5. Date: Mon, 08 Apr 1996 11:04:30 -0700
  6. Organization: Apple Computer, Inc., Cupertino, California
  7. Message-ID: <3169552E.B49@bayarea.net>
  8. References: <4k4gja$mm@nuscc.nus.sg>
  9. NNTP-Posting-Host: a17-128-200-53.apple.com
  10. Mime-Version: 1.0
  11. Content-Type: text/plain; charset=us-ascii
  12. Content-Transfer-Encoding: 7bit
  13. X-Mailer: Mozilla 2.01 (Macintosh; I; PPC)
  14.  
  15. Hi
  16.  
  17. I just stopped on the first major prob I found.
  18.  
  19. Virus wrote:
  20. > ...
  21. >
  22. > char* SpaceStudentFile::readNextField()
  23. > {
  24. >   char str[40];
  25. >   int i = 0;
  26. > 
  27. >   // take away starting whitespaces
  28. >   while (str[0] = fgetc(fd)) {
  29. >         if (*str == ' ') continue;
  30. >         if (*str == '\t') continue;
  31. >         if (*str == '\n') continue;
  32. >         if (*str == EOF) return NULL;
  33. >         break;
  34. >   }
  35. > 
  36. >   // read from " to next "
  37. >   if (str[0] == '"') {
  38. >         for (i=0; ((str[i] = fgetc(fd)) != '"'); i++)
  39. >         ;
  40. >         str[i]=0;
  41. >         return str;
  42. >   }
  43. > 
  44. >   // else read until next whitespace
  45. >   for (i=1; ((str[i] = fgetc(fd)) != ' ')
  46. >         && (str[i] != '\n') && (str[i] != '\t'); i++)
  47. >   ;
  48. >   str[i] = 0;
  49. >   return str;
  50. > 
  51. > }
  52. > ....
  53.  
  54. str is a local variable, that means it is created when you
  55. call this method, and removed when the function returns to its
  56. caller, so when you do "return str" you actually return a pointer
  57. to a non existing string.
  58. There are several valid solutions, one of them is to allocate
  59. memory for it like:
  60.  
  61. char* SpaceStudentFile::readNextField()
  62. {
  63.  char str[40];
  64.  char * newStr;
  65.  
  66.  ....
  67.  
  68.  str[i] = 0;
  69.  newStr = strdup(str); // If your system doesn't have strdup you can use
  70.                        // newStr = malloc(strlen(str) + 1);
  71.                        // strcpy(newStr, str);
  72.  return newStr;
  73. }
  74.  
  75. This is a simple solution that will work, but this is not a very
  76. good one (you need to remember to free the memory when you
  77. don't need it anymore, and this is the C way, not C++). If you have 
  78. some time to spend on this, than try to use in your classes the String 
  79. class instead of "char *"
  80.  
  81. Amir
  82.  
  83.  
  84. -- 
  85.  
  86.                     \\||||//
  87.                     \ o  o /
  88. ______________.ooO__(  ()  )__Ooo._______________________________
  89.  
  90.    Amir Deutel                  http://www.bayarea.net/~amir/
  91.  
  92.                  .oo0O   O0oo.  Work: Amir.D@AppleLink.Apple.com
  93.                  (   )   (   )  Home: Amir@bayarea.net
  94. __________________\ (_____) /____________________________________
  95.                    \_)   (_/
  96.